Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

misc: added cause to unsupported signing algorithm exception #1036

Merged
merged 4 commits into from
Feb 15, 2024

Conversation

0marperez
Copy link
Contributor

Issue #

N/A

Description of changes

-Added the cause constructor parameter to the UnsupportedSigningAlgorithmException.
-Fixed outdated documentation.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@0marperez 0marperez added the no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly. label Feb 12, 2024
@0marperez 0marperez marked this pull request as ready for review February 13, 2024 16:43
@0marperez 0marperez requested a review from a team as a code owner February 13, 2024 16:43
@@ -12,10 +12,14 @@ import aws.smithy.kotlin.runtime.InternalApi
*
* See: [AwsSigningAlgorithm], [AwsSigner]
*
* @param isSigV4a Whether the unsupported signing algorithm is sigV4a
* @param signingAlgorithm The unsupported signing algorithm
Copy link
Contributor

@lauzadis lauzadis Feb 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing @param message, @param cause

Copy link

sonarcloud bot commented Feb 14, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@0marperez 0marperez merged commit fcde0c3 into main Feb 15, 2024
13 checks passed
@0marperez 0marperez deleted the mrap-link-docs branch February 15, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Indicates that a changelog entry isn't required for a pull request. Use sparingly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants